Skip to content

docs(preprod): Document skipped status checks - #18845

Merged
jamieQ merged 5 commits into
masterfrom
docs/eme-1218-skip-status-check-guidance
Jul 31, 2026
Merged

docs(preprod): Document skipped status checks#18845
jamieQ merged 5 commits into
masterfrom
docs/eme-1218-skip-status-check-guidance

Conversation

@jamieQ

@jamieQ jamieQ commented Jul 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

DESCRIBE YOUR PR

Documents how teams can satisfy required Size Analysis and Snapshot Testing
checks when CI workflows intentionally skip artifact uploads.

Refs EME-1218
Refs EME-1258

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.)
  • Other deadline
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@jamieQ
docs(preprod): Document skipped status checks
21f16ec 
Explain how CI workflows can satisfy required checks when they intentionally skip artifact uploads.

Refs EME-1218
@linear-code

linear-code Bot commented Jul 27, 2026 •
edited
Loading

Copy link
Copy Markdown

EME-1218

EME-1258

@vercel

vercel Bot commented Jul 27, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview Jul 31, 2026 2:18pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview Jul 31, 2026 2:18pm

Request Review

@jamieQ
docs(preprod): Move skip guidance outside setup flow
5d58ff2 
Keep the numbered integration steps focused on the primary setup path and present optional status-check skipping as a separate section.

Refs EME-1218
@jamieQ
jamieQ marked this pull request as ready for review July 27, 2026 20:38
@jamieQ
jamieQ requested a review from mtopo27 July 27, 2026 20:53
coolguyzone
coolguyzone approved these changes Jul 29, 2026
View reviewed changes

@coolguyzone coolguyzone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for updating!

@jamieQ
docs(preprod): Link skipped status check API docs
c27010b 
Remove the outdated experimental notices now that the endpoints are public.

Refs EME-1218
@jamieQ
jamieQ merged commit ad6cf4d into master Jul 31, 2026
22 checks passed
@jamieQ
jamieQ deleted the docs/eme-1218-skip-status-check-guidance branch July 31, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coolguyzone coolguyzone coolguyzone approved these changes
@mtopo27 mtopo27 Awaiting requested review from mtopo27

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jamieQ @coolguyzone